From: Corey Farwell Date: Fri, 6 May 2016 01:36:13 +0000 (-0400) Subject: Utilize `if let` over single branch `match`. X-Git-Tag: archive/raspbian/0.35.0-2+rpi1~3^2^2^2^2^2^2^2~22^2~14^2~26^2 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/success//%22http:/www.example.com/cgi/success/?a=commitdiff_plain;h=f1062ce338370e19109cee0a922baa4398b0c816;p=cargo.git Utilize `if let` over single branch `match`. --- diff --git a/src/cargo/core/package_id_spec.rs b/src/cargo/core/package_id_spec.rs index a643713b2..031249393 100644 --- a/src/cargo/core/package_id_spec.rs +++ b/src/cargo/core/package_id_spec.rs @@ -17,14 +17,12 @@ pub struct PackageIdSpec { impl PackageIdSpec { pub fn parse(spec: &str) -> CargoResult { if spec.contains("/") { - match spec.to_url() { - Ok(url) => return PackageIdSpec::from_url(url), - Err(..) => {} + if let Ok(url) = spec.to_url() { + return PackageIdSpec::from_url(url); } if !spec.contains("://") { - match Url::parse(&format!("cargo://{}", spec)) { - Ok(url) => return PackageIdSpec::from_url(url), - Err(..) => {} + if let Ok(url) = Url::parse(&format!("cargo://{}", spec)) { + return PackageIdSpec::from_url(url); } } }